Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support scheduling Cloud Agent on master nodes #164

Merged
merged 1 commit into from
Jul 7, 2023

Conversation

DebakelOrakel
Copy link
Contributor

@DebakelOrakel DebakelOrakel commented Jul 7, 2023

Checklist

  • The PR has a meaningful title. It will be used to auto generate the
    changelog.
    The PR has a meaningful description that sums up the change. It will be
    linked in the changelog.
  • PR contains a single logical change (to build a better changelog).
  • Update the documentation.
  • Categorize the PR by adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog.
  • Link this PR to related issues or PRs.

@DebakelOrakel DebakelOrakel added the enhancement New feature or request label Jul 7, 2023
@DebakelOrakel DebakelOrakel requested a review from a team as a code owner July 7, 2023 12:51
@DebakelOrakel DebakelOrakel linked an issue Jul 7, 2023 that may be closed by this pull request
class/defaults.yml Outdated Show resolved Hide resolved
component/main.jsonnet Outdated Show resolved Hide resolved
@DebakelOrakel DebakelOrakel force-pushed the feat/schedule-on-master branch from d72611a to 91ee54a Compare July 7, 2023 12:56
@DebakelOrakel
Copy link
Contributor Author

@simu Do you by chance have insight, what jsonnet linter is unhappy whit?

@simu
Copy link
Member

simu commented Jul 7, 2023

@simu Do you by chance have insight, what jsonnet linter is unhappy whit?

make format spits out the following diff:

diff --git a/component/main.jsonnet b/component/main.jsonnet
index fa73d86..1672148 100644
--- a/component/main.jsonnet
+++ b/component/main.jsonnet
@@ -10,7 +10,7 @@ local params = inv.parameters.appuio_cloud;
 local isOpenshift = std.startsWith(inv.parameters.facts.distribution, 'openshift');
 local namespaceLabels = (
   if isOpenshift then { 'openshift.io/cluster-monitoring': 'true' }
-  else { 'SYNMonitoring': 'main' }
+  else { SYNMonitoring: 'main' }
 ) + params.namespaceLabels;
 local namespaceAnnotations = (
   if isOpenshift then { 'openshift.io/node-selector': '' }

@DebakelOrakel DebakelOrakel force-pushed the feat/schedule-on-master branch from 91ee54a to 4b9ffcf Compare July 7, 2023 13:04
@DebakelOrakel DebakelOrakel force-pushed the feat/schedule-on-master branch from 4b9ffcf to 2e0a333 Compare July 7, 2023 13:05
@DebakelOrakel DebakelOrakel requested a review from simu July 7, 2023 13:06
@DebakelOrakel DebakelOrakel merged commit 8a60fa4 into master Jul 7, 2023
@DebakelOrakel DebakelOrakel deleted the feat/schedule-on-master branch July 7, 2023 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Schedule Cloud Agent on master nodes
2 participants